]> dgit.raspbian.org Git - gnome-shell.git/commitdiff
switcherPopup: Use MonitorConstraint instead of vfunc overrides
authorGeorges Basile Stavracas Neto <georges.stavracas@gmail.com>
Thu, 23 Aug 2018 16:35:43 +0000 (13:35 -0300)
committerGeorges Basile Stavracas Neto <georges.stavracas@gmail.com>
Tue, 9 Oct 2018 01:42:23 +0000 (22:42 -0300)
Instead of overriding vfunc_get_preferred_width|height(), use the
already available Layout.MonitorConstraint to bind SwitcherPopup
to the primary monitor.

js/ui/switcherPopup.js

index 4b1d89e766dbf80e3d6b85682639dafa92d90b3a..ad77787cfb3e376510de0059dc15495b3d312522 100644 (file)
@@ -10,6 +10,7 @@ const Shell = imports.gi.Shell;
 const Signals = imports.signals;
 const St = imports.gi.St;
 
+const Layout = imports.ui.layout;
 const Main = imports.ui.main;
 const Tweener = imports.ui.tweener;
 
@@ -63,21 +64,13 @@ var SwitcherPopup = new Lang.Class({
         this._initialDelayTimeoutId = 0;
         this._noModsTimeoutId = 0;
 
+        this.add_constraint(new Layout.MonitorConstraint({ primary: true }));
+
         // Initially disable hover so we ignore the enter-event if
         // the switcher appears underneath the current pointer location
         this._disableHover();
     },
 
-    vfunc_get_preferred_width(forHeight) {
-        let primary = Main.layoutManager.primaryMonitor;
-        return [primary.width, primary.width];
-    },
-
-    vfunc_get_preferred_height(forWidth) {
-        let primary = Main.layoutManager.primaryMonitor;
-        return [primary.height, primary.height];
-    },
-
     vfunc_allocate(box, flags) {
         this.set_allocation(box, flags);